Skip to content

fix(macos): enhance deployment and packaging for macOS#2158

Open
theangelperalta wants to merge 2 commits intolem-project:mainfrom
theangelperalta:fix/macos-deployment-improvements
Open

fix(macos): enhance deployment and packaging for macOS#2158
theangelperalta wants to merge 2 commits intolem-project:mainfrom
theangelperalta:fix/macos-deployment-improvements

Conversation

@theangelperalta
Copy link
Copy Markdown
Contributor

@theangelperalta theangelperalta commented May 1, 2026

Improves macOS deployment and packaging:

  • Icon support: Added make icns target to generate macOS .icns icon from PNG, and copy it into the app bundle during deployment
  • Platform-aware install: make install now skips Linux desktop integration steps on macOS
  • Tree-sitter deploy fix: Excludes optional tree-sitter libraries from bundling since they are loaded at runtime when available

- Add automatic .icns icon generation from PNG in Makefile
- Skip desktop integration install on macOS (Darwin-specific)
- Copy .icns file to app bundle during deployment
- Configure deploy library to exclude optional tree-sitter libraries
  from bundling (they're loaded at runtime when available)

These changes improve macOS app packaging by:
1. Generating proper .icns icons for dock/taskbar display
2. Avoiding Linux-specific desktop file installations on macOS
3. Preventing tree-sitter library bundling issues on macOS
@code-contractor-app
Copy link
Copy Markdown
Contributor

code-contractor-app Bot commented May 1, 2026

✅ Code Contractor Validation: PASSED

🎉 All violations have been dismissed.

ℹ️ 1 violation(s) dismissed

📚 About Code Contractor

Declarative Code Standards That Learn and Improve

Define domain-specific validation rules in YAML.
Your contracts document team knowledge and evolve into more accurate AI enforcement.

Want this for your repo?
Install Code Contractor

Copy link
Copy Markdown
Contributor

@code-contractor-app code-contractor-app Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Contractor validation failed ❌ — see the sticky comment for full results.

Comment thread src/macosx.lisp Outdated
@code-contractor-app code-contractor-app Bot dismissed their stale review May 1, 2026 22:47

All violations have been dismissed via resolved conversations.

Copy link
Copy Markdown
Contributor

@code-contractor-app code-contractor-app Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ All violations have been dismissed. This PR is ready to merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant